Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add African pidgin language translation #3156

Merged
merged 9 commits into from
Jan 13, 2024

Conversation

LizzyOrji123
Copy link
Contributor

@LizzyOrji123 LizzyOrji123 commented Oct 22, 2023

I created a new language called pidgin and created best-practice.md and other files.

The goal of my branch is to encourage Africans to join the open source community by writing in a language every African can understand which is Pidgin. I would love to recreate the entire guidelines in Pidgin. Thank you.… added all best practices in pidgin language

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


… added all best practices in pidgin language
@LizzyOrji123 LizzyOrji123 requested a review from a team as a code owner October 22, 2023 18:13
@LizzyOrji123 LizzyOrji123 temporarily deployed to Pages Preview October 22, 2023 18:14 — with GitHub Actions Inactive
@LizzyOrji123
Copy link
Contributor Author

LizzyOrji123 commented Oct 22, 2023

If you speak pidgin and would love to contribute to the pidgin open-source guide on Github. I would appreciate your help.

@LizzyOrji123 LizzyOrji123 temporarily deployed to Pages Preview October 24, 2023 12:06 — with GitHub Actions Inactive
Copy link
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thank you for the PR! I am happy to see this language get added. There are a few changes it needs before we can merge though.

  • The directory should be called pcm instead of pidgin , this is the ISO designation for the language: https://iso639-3.sil.org/code/pcm
  • Please rename the language file in the _data directory to be pcm.yml and change the locale_name
  • Add the rest of the untranslated pages into the pcm directory as copies from the English version.

_data/locales/pidgin.yml Outdated Show resolved Hide resolved
_data/locales/pidgin.yml Outdated Show resolved Hide resolved
@ahpook ahpook changed the title created a new language called pidgin and created best-practice.md and… The goal of my branch is to encourage Africans to join the open source community by writing in a language every African can understand which is Pidgin. I would love to recreate the entire guidelines in Pidgin. Thank you. Add African pidgin language translation Nov 3, 2023
@ahpook
Copy link
Contributor

ahpook commented Dec 6, 2023

@LizzyOrji123 Hi, are you still interested in making this contribution? I left a review with a few requested changes - I'd love to get this merged! Please take a look.

@LizzyOrji123
Copy link
Contributor Author

@LizzyOrji123 Hi, are you still interested in making this contribution? I left a review with a few requested changes - I'd love to get this merged! Please take a look.

I have made the corrections. Thank you for being so patient and detailed with your work.

Copy link
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great @LizzyOrji123 ! Thanks so much. I made a couple of small suggestions in code review so that the metadata will work with the site generator - I think once those changes are in we'll be good to merge!

_data/locales/pcm.yml Outdated Show resolved Hide resolved
assets/css/translate.scss Outdated Show resolved Hide resolved
_articles/pcm/code-of-conduct.md Outdated Show resolved Hide resolved
_articles/pcm/finding-users.md Outdated Show resolved Hide resolved
- Markdown style wants `*` not `-` for lists
- Removed some extra blank lines
- The `retext-repeated-words` plugin has only a small list of
  words that are allowed to be repeated, but pidgin uses doubled
  words as intensifiers ("small small" for "very small"). I added
  hyphens ("small-small") because I saw some examples of that
- Renamed the `best-practice.md` -> `best-practices.md` to match
  the rest of the site
- Copied the untranslated articles in so the site build works
- The missing `index.html` in this directory was preventing
  the site build from completely working
- Renamed Internal anchor links like `#sabi-to-say-no` to match
  the new section header names
- I'd blithely copied the untranslated files in but not changed
  their front matter to match the directory name
Weird, this one didn't come up locally
@ahpook ahpook added this pull request to the merge queue Jan 13, 2024
Merged via the queue into github:main with commit aef2cc2 Jan 13, 2024
3 checks passed
@ahpook
Copy link
Contributor

ahpook commented Jan 13, 2024

Very happy to get this merged in. This is now LIVE on the site! You can see it here

https://opensource.guide/pcm/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants